Use TokenPaginatedTable for Component Search#438
Merged
Conversation
Replace the legacy bootstrap-table with TokenPaginatedTable and update table props/behaviour to use token-based pagination.
5 tasks
nscuro
requested changes
Mar 23, 2026
nscuro
requested changes
Mar 30, 2026
Up to standards ✅🟢 Issues
|
Member
|
Not sure if that was an issue before, but when clicking the search button I'm getting these warnings: |
nscuro
reviewed
Apr 14, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Migrates the Portfolio Component Search view from legacy bootstrap-table server pagination to the shared TokenPaginatedTable component, aligning the frontend with the backend’s token-based pagination API (Issue #2049 / hyades-apiserver#1867).
Changes:
- Replaced
<bootstrap-table>usage with<token-paginated-table>and switched to/api/v2/componentswith query-param based filtering + sorting. - Updated search UI/logic (notably HASH search now includes hash-type selection) and refactored query param construction to an object-based approach.
- Simplified
hashes_short_desctranslations across multiple locales.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| src/views/portfolio/components/ComponentSearch.vue | Migrates component search results to TokenPaginatedTable; updates filters, sorting, and HASH search UI. |
| src/i18n/locales/en.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/de.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/es.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/fr.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/hi.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/it.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/ja.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/pl.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/pt.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/pt-BR.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/ru.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/uk-UA.json | Simplifies hashes_short_desc string. |
| src/i18n/locales/zh.json | Simplifies hashes_short_desc string. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
nscuro
changed the title
TokenPaginatedTable for Component SearchTokenPaginatedTable for Component Search
nscuro
approved these changes
Apr 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Replace the legacy bootstrap-table with TokenPaginatedTable and update table props/behaviour to use token-based pagination.
Addressed Issue
Issue: DependencyTrack/hyades#2049
Backend: DependencyTrack/hyades-apiserver#1867
Additional Details
Checklist